Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ember Update #6

Merged
merged 3 commits into from
May 13, 2019
Merged

Ember Update #6

merged 3 commits into from
May 13, 2019

Conversation

simonihmig
Copy link
Contributor

No description provided.

@rwjblue
Copy link
Member

rwjblue commented May 13, 2019

FWIW, using the modern test style is quite hard on Ember < 2.4 (the new APIs heavily depend on this.owner and "owner" as a concept didn't land until 2.4ish).

@rwjblue
Copy link
Member

rwjblue commented May 13, 2019

Since this is a breaking change anyways (Node version at least), I'd personally drop all versions less than 2.18 (over 1.5 years old at this point!) but 2.12 could work too (which is > 2 years old).

@simonihmig
Copy link
Contributor Author

I settled on keeping support for 2.12, as having -in-element available is what we care about here. Fyi.

@simonihmig simonihmig merged commit 4441ce9 into master May 13, 2019
@simonihmig simonihmig deleted the update branch May 13, 2019 22:16
simonihmig added a commit that referenced this pull request May 13, 2019
Not needed anymore since dropping support for Ember <2.12 in #6
@simonihmig simonihmig mentioned this pull request May 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants